Skip to content

Superbenchmark executor: online inference config file and custom repo link#369

Closed
LiYueqian-James wants to merge 8 commits into
microsoft:mainfrom
LiYueqian-James:yueqianli/SBConfigAndRepo
Closed

Superbenchmark executor: online inference config file and custom repo link#369
LiYueqian-James wants to merge 8 commits into
microsoft:mainfrom
LiYueqian-James:yueqianli/SBConfigAndRepo

Conversation

@LiYueqian-James

Copy link
Copy Markdown
Contributor

Added support for

  1. Feeding an online inference config file as a link, without the need to zip into packages and upload to storage account
  2. Feeding a different superbenchmark repo link other the default microsoft SB repo. Some users have added more benchmark to their own version of superbench and can use this method run them in VC.

/// <summary>
/// Link to the superbench repo.
/// </summary>
public string RepositoryLink

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to change the repository link? Even if we need to support extra config?

string configFileName = Path.GetFileName(configFileUri.AbsolutePath);
string configFullPath = this.PlatformSpecifics.Combine(this.SuperBenchmarkDirectory, configFileName);

using (var client = new HttpClient())

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's not add un-unit testable code here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are cmd line based methods preferred? like wget or curl?

@yangpanMS

Copy link
Copy Markdown
Contributor

Could we do this instead: you are allowed to use extra dependency (for example git clone) in the profile, and you give it a package name. You then could reference the file you cloned by path.

@LiYueqian-James

Copy link
Copy Markdown
Contributor Author

Could we do this instead: you are allowed to use extra dependency (for example git clone) in the profile, and you give it a package name. You then could reference the file you cloned by path.

so it's not just about inference config file. some people have created their own SB benchmarks (in their own repo) and want to run those. The official one do not have these benchmarks. That's why I added a parameter for repoLink

@yangpanMS yangpanMS closed this Jun 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants